Skip to content

Comments

모집 마감 처리#392

Merged
HA-SEUNG-JEONG merged 2 commits intomainfrom
fix/group-study
Feb 11, 2026
Merged

모집 마감 처리#392
HA-SEUNG-JEONG merged 2 commits intomainfrom
fix/group-study

Conversation

@HA-SEUNG-JEONG
Copy link
Contributor

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Feb 11, 2026

🌱 연관된 이슈

☘️ 작업 내용

  • 스터디 링크 로직 try-catch문으로 강화
  • 마감된 스터디의 경우 모집 마감으로 텍스트 변경

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

  • 새로운 기능
    • 모집 마감일 자동 인식: 공고 시작일/마감 상태를 판별해 마감된 경우 신청 버튼을 비활성화하고 버튼 텍스트를 '모집 마감'으로 표시합니다.
    • 개선된 URL 공유 알림: 네이티브 alert 대신 토스트 알림으로 변경하여 URL 복사 성공 및 실패를 화면에 명확히 안내합니다.

@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-platform-client-dev Ready Ready Preview, Comment Feb 11, 2026 4:19am

@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Feb 11, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

스터디 정보 요약 컴포넌트에서 URL 복사 피드백을 토스트로 전환하고, 시작일을 기준으로 신청 마감 여부를 판단해 버튼 비활성화 및 '모집 마감' 텍스트 표시 로직을 추가했습니다.

Changes

Cohort / File(s) Summary
Deadline Logic & Toast Integration
src/components/summary/study-info-summary.tsx
클립보드 복사 성공/실패 알림을 useToastStore/showToast로 교체하고, isDeadlinePassed 플래그를 도입해 시작일 기반 마감 판단을 추가함. 신청 버튼 활성화 조건 및 버튼 텍스트 로직에 마감 여부 반영.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 토스트를 꾹 눌러 소식 전해요,
복사 빛나는 순간 뽁하고 알려주고,
달이 지나면 문은 살짝 닫혀요,
모집은 멈추고 버튼은 쉬어요,
당근 한 입에 축하의 춤을 출게요 🥕🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목이 변경사항의 주요 내용과 일치합니다. 마감된 스터디 처리라는 핵심 변경사항을 명확하게 요약하고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/group-study

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
src/components/summary/study-info-summary.tsx (1)

123-129: 클립보드 미지원 환경에 대한 가드 추가를 고려하세요.
권한/지원 미비 환경에서 불필요한 예외를 던지기보다 즉시 안내하는 편이 UX가 더 명확합니다.

✅ 제안 수정안
 const handleCopyURL = async () => {
+  if (!navigator?.clipboard) {
+    showToast('클립보드 복사를 지원하지 않는 환경입니다.', 'error');
+    return;
+  }
   try {
     await navigator.clipboard.writeText(window.location.href);
     showToast('스터디 링크가 복사되었습니다!');
   } catch {
     showToast('클립보드 복사에 실패했습니다. 다시 시도해주세요.', 'error');
   }
 };

Comment @coderabbitai help to get the list of available commands and usage tips.

@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit 783ed78 into main Feb 11, 2026
9 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the fix/group-study branch February 11, 2026 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant